feat(linux/kwin): log object serial when available on stream creation#5299
Merged
ReenigneArcher merged 1 commit intoJun 23, 2026
Merged
Conversation
a760475 to
d418a27
Compare
d418a27 to
3ada381
Compare
3ada381 to
ca85f9c
Compare
Bundle ReportBundle size has no change ✅ |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5299 +/- ##
==========================================
- Coverage 17.85% 17.85% -0.01%
==========================================
Files 111 111
Lines 24602 24605 +3
Branches 10892 10894 +2
==========================================
Hits 4393 4393
+ Misses 17674 16819 -855
- Partials 2535 3393 +858
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 51 files with indirect coverage changes Continue to review full report in Codecov by Harness.
|
|
vindeckyy
pushed a commit
to vindeckyy/Solar-Flare
that referenced
this pull request
Jun 27, 2026
…LizardByte#5299) (cherry picked from commit fdf1363)
vindeckyy
added a commit
to vindeckyy/Solar-Flare
that referenced
this pull request
Jun 27, 2026
…ted patch
Third round of fork improvements on top of the branding/CI work:
* docs/CHANGELOG-SolarFlare.md (145 lines). Fork-specific changelog
separate from upstream's docs/changelog.md. Enumerates every fork
change (config plumbing, branding, web UI, build script, docs,
cherry-picked upstream commits) in the unreleased section. Cross-
references docs/PORTING.md, docs/CONFIGURATION.md, README.md and
PUSH-INSTRUCTIONS.md.
* scripts/cachyos-build.sh post-install verification. After the
'Install + verification' step, the script now:
1. Verifies the installed sunshine binary contains the fork
banner ('Fork: SolarFlare') -- catches the case where a
leftover upstream package overwrote the fork install.
2. Writes a 5-key test config to /tmp, runs 'sunshine <conf>'
for 1 second, and checks that all 5 fork keys appear in the
parse log without any 'Unrecognized' warnings.
Both checks are non-fatal warnings (so the script still succeeds
if sunshine isn't on $PATH or the binary is older than the
source), but they're loud enough that 'I just installed the
fork and a key isn't taking effect' is now a clearly printed
warning instead of a silent runtime surprise.
Also updated the post-install Done banner to identify the binary
as 'SolarFlare' and surface the 5 fork tunables (was 'Sunshine').
* Regenerated cachyos-fastpath.patch against upstream base commit
1fce18d, including the cumulative fork changes (config keys,
branding macro). Was 420 lines / 7 files / +280/-2; now 569 lines
/ 7 files / +379/-2. Verified to apply cleanly to a fresh clone
of LizardByte/Sunshine HEAD with 3-way merge for upstream's own
changes to pipewire.cpp / stream.cpp since the fork base.
* PUSH-INSTRUCTIONS.md updated to:
- Reflect the new patch stats (+379/-2, was +280/-2).
- Show the regeneration command for both the cumulative patch
(git diff 1fce18d..HEAD) and the historical patch
(git diff 1fce18d..bbcd69b -> cachyos-fastpath-historical.patch).
- Document the 3-way merge caveat for pipewire.cpp / stream.cpp.
- Cross-reference docs/CHANGELOG-SolarFlare.md.
* Also cherry-picked in this batch (already committed locally):
- 838aa57 build(deps): fix building on Linux with DRM capture disabled (LizardByte#5224)
- 56d5936 feat(linux/xdgportal): Add support for pipewire-serial (LizardByte#5060)
- eebb1b0 feat(linux/kwin): log object serial when available on stream creation (LizardByte#5299)
- 69a2db2 fix(crypto): OpenSSL 4.x compatibility (LizardByte#5330)
All four were verified to not touch any fork-modified file, so
cherry-pick was conflict-free.
Verification:
* cmake --build build: success (incremental).
* ./build/tests/test_sunshine: 334 tests, 329 passed (was 328; the
new upstream test_crypto.cpp added 1 test), 5 pre-existing skipped,
0 failed. Exit 0.
* bash -n scripts/cachyos-build.sh: OK (post-install verification
block has valid shell syntax).
* git apply --check cachyos-fastpath.patch on a fresh clone of
LizardByte/Sunshine HEAD: applies cleanly (0 errors).
* Dependabot: 0 open alerts (still clean from round 2).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Description
Log the pipewire object serial as well as the node when a valid one is received from KDE's screencasting wayland extension to be consistent with other debug logging.
Screenshot
Issues Fixed or Closed
Roadmap Issues
Type of Change
Checklist
AI Usage